Skip to content

Conversation

@ilandikov
Copy link
Collaborator

@ilandikov ilandikov commented Dec 4, 2025

Types of changes

Done by pairing with @claremacrae

Changes visible to users:

Description

Prior to this PR, when the Toolbar input 'Filter by description' was used, as soon as the search was redrawn the filter was lost, and all matching tasks were shown.

This meant that if you wanted to narrow down a Tasks code block to find-and-edit several tasks, you had to re-enter the search term after every edit.

And that got very old, very quickly!

Now, any filter string in the Toolbar is now retained.

Note that any cursor position is, however, not retained. We don't think that is a high priority to fix.

Motivation and Context

This is a critical component of making #3693 releasable...

How has this been tested?

  • Unit test + manual test

Checklist

  • My code follows the code style of this project and passes yarn run lint.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • My change has adequate Unit Test coverage.
    • The filter string being kept in the search box between task rerenders is tested manually

Terms

@claremacrae claremacrae changed the title Fix/results filtered correctly fix: Results filter is now re-applied when settings or tasks are edited Dec 4, 2025
@claremacrae claremacrae added scope: rendering of tasks How the plugin displays tasks, including search results (except CSS issues) scope: user interface UI other than Edit task modal, e.g. context menus, date picker labels Dec 4, 2025
@claremacrae claremacrae changed the title fix: Results filter is now re-applied when settings or tasks are edited fix: Results filter now re-applied when settings or tasks edited Dec 4, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 4, 2025

Copy link
Collaborator

@claremacrae claremacrae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really like this!

@claremacrae claremacrae merged commit b4e8731 into obsidian-tasks-group:main Dec 4, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: rendering of tasks How the plugin displays tasks, including search results (except CSS issues) scope: user interface UI other than Edit task modal, e.g. context menus, date picker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants